Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[HOTFIX]날짜 필터링 검증 버그 해결 #255

Merged
merged 3 commits into from
May 31, 2024

Conversation

kmw2378
Copy link
Contributor

@kmw2378 kmw2378 commented May 29, 2024

#️⃣연관된 이슈

close #254

📝작업 내용

  • Date VO의 endDateLocalDate.now() 보다 이후인 경우만 검증하도록 수정
  • startDate, endDate 둘 중 하나라도 null 이면 예외를 발생시키도록 수정
    • 현재는 두 개 모두 null이여야 예외가 발생하도록 구현되있음

LocalDate.isAfter()가 같은날인 경우도 false를 반환하여 발생한 버그를 해결하였습니다.

@kmw2378 kmw2378 added bug 버그입니다. 🖤back 백엔드 업무입니다. refactor 코드가 내부적으로 작동하는 방식을 변경하되, 변경 전과 동일하게 작동합니다. labels May 29, 2024
@kmw2378 kmw2378 requested review from YeaChan05 and hwan2-99 May 29, 2024 15:32
@kmw2378 kmw2378 self-assigned this May 29, 2024
@kmw2378 kmw2378 changed the title [HOTFIX]날짜 필터링 검증시 오늘날까지 조회인 경우 허용하도록 로직 수정 [HOTFIX]날짜 필터링 검증 버그 해결 May 29, 2024
@kmw2378 kmw2378 enabled auto-merge May 31, 2024 02:59
@kmw2378 kmw2378 disabled auto-merge May 31, 2024 03:20
@kmw2378 kmw2378 enabled auto-merge May 31, 2024 03:20
@kmw2378 kmw2378 merged commit 401f55b into KakaoFunding:develop May 31, 2024
1 check passed
@kmw2378 kmw2378 deleted the bug/254-date-filter-end-date branch May 31, 2024 03:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🖤back 백엔드 업무입니다. bug 버그입니다. refactor 코드가 내부적으로 작동하는 방식을 변경하되, 변경 전과 동일하게 작동합니다.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants